Closed
Bug 1412769
Opened 8 years ago
Closed 8 years ago
nsSmtpProtocol::SendRecipientResponse(): Identical code for different branches
Categories
(MailNews Core :: Networking: SMTP, enhancement)
MailNews Core
Networking: SMTP
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 58.0
People
(Reporter: Sylvestre, Assigned: Sylvestre)
References
(Blocks 1 open bug)
Details
(Keywords: coverity, Whiteboard: [CID 1420380])
Attachments
(1 file)
|
1.60 KB,
patch
|
jorgk-bmo
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Comment 1•8 years ago
|
||
Attachment #8923298 -
Flags: review?(jorgk)
|
||
Comment 2•8 years ago
|
||
Comment on attachment 8923298 [details] [diff] [review]
aze.diff
Nice find ;-) How did you find this? Some linting or clang trick?
Please let me land this for you!! I'm coordinating our landings with M-C merges.
Attachment #8923298 -
Flags: review?(jorgk) → review+
|
|
Assignee | |
Comment 4•8 years ago
|
||
Coverity - https://scan.coverity.com/projects/thunderbird :)
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/55b85200c486
nsSmtpProtocol::SendRecipientResponse(): Remove identical code for if/else branches. r=jorgk
|
|
||
Updated•8 years ago
|
Target Milestone: --- → Thunderbird 58.0
You need to log in
before you can comment on or make changes to this bug.
Description
•